Skip to content

Conversation

yashovardhan
Copy link
Member

Adds the following:

  • Adds playground based on Web3Auth Quick Start
  • Add highlighting and blur magic comments
  • Update home page
  • Update nav bar components

Max time was spent on maintaining the same design scheme, although a design review will be ideal

@yashovardhan yashovardhan requested review from a team as code owners July 28, 2025 09:47
Copy link

vercel bot commented Jul 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
metamask-docs Ready Ready Preview Comment Sep 2, 2025 8:16am

Copy link

socket-security bot commented Jul 28, 2025

cursor[bot]

This comment was marked as outdated.

@yashovardhan yashovardhan changed the title POC for new flow of documentation New flow of documentation Sep 2, 2025
bgravenorst
bgravenorst previously approved these changes Sep 2, 2025
Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

ihsraham
ihsraham previously approved these changes Sep 2, 2025
bgravenorst
bgravenorst previously approved these changes Sep 2, 2025
Copy link
Member

@shahbaz17 shahbaz17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Comment on lines -1 to -4
{
"label": "Tutorials",
"position": 5
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link to global tutorial is missing like SDK. We can either push it in the PR, happy if we are fixing with a separate PR. Not a blocker.
Screenshot 2025-09-02 at 3 27 02 PM

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dtk sidebar is autogenerated. You have manually make the sidebar in order to add links

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexandratran have created another issue to tackle this. #2263

Copy link
Member

@AyushBherwani1998 AyushBherwani1998 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a minor comment.

@yashovardhan yashovardhan merged commit f7c521c into main Sep 2, 2025
13 of 19 checks passed
@yashovardhan yashovardhan deleted the feat/quick-start branch September 2, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants